The storage.view_settings() getter returns a reference, so cloning the entire BTreeMap of view settings here is unnecessary and introduces redundant allocations. You can pass the reference directly.

		document.apply_stored_document_settings(storage.view_settings());

Cloning the entire NodeNetwork here is unnecessary and expensive because network is only passed by reference to network_ids, stage_runtime_snapshot, and verify_storage_round_trip. You can borrow the document network directly without cloning.

		let network = self.network_interface.document_network();

Cloning the path vector and pushing it to the stack for every single node (including leaf nodes) is highly inefficient. Since only nodes that represent nested networks actually need to be traversed, you can filter the keys to only those where network_metadata is Some before cloning the path. This avoids redundant vector allocations and map lookups for all leaf nodes.

			stack.extend(
				self_network_metadata
					.persistent_metadata
					.node_metadata
					.iter()
					.filter(|(_, meta)| meta.persistent_metadata.network_metadata.is_some())
					.map(|(node_id, _)| {
						let mut current_path = path.clone();
						current_path.push(*node_id);
						current_path
					}),
			);